Skip to content

Update docs to demonstrate specifying Qdrant version#917

Merged
cristianrgreco merged 5 commits intotestcontainers:mainfrom
Anush008:qdrant-version
Mar 6, 2025
Merged

Update docs to demonstrate specifying Qdrant version#917
cristianrgreco merged 5 commits intotestcontainers:mainfrom
Anush008:qdrant-version

Conversation

@Anush008
Copy link
Copy Markdown

@Anush008 Anush008 commented Mar 5, 2025 •
edited
Loading

Hey 👋. Greetings from Qdrant.

QdrantContainer defaults to a specific version of Qdrant.

We had users coming from https://node.testcontainers.org/modules/qdrant/ not knowing how to specify a custom version. So this PR helps with demonstrating that.

Anush008 added 2 commits March 5, 2025 13:54
@Anush008
docs: Demonstrate Qdrant version
f43b9d1 
Signed-off-by: Anush008 <anushshetty90@gmail.com>
@Anush008
chore: Bump Qdrant version
4131fe4 
Signed-off-by: Anush008 <anushshetty90@gmail.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 5, 2025 •
edited
Loading

✅ Deploy Preview for testcontainers-node ready!

Name Link
🔨 Latest commit d5197b5
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-node/deploys/67c8526dcf5a170008f543e9
😎 Deploy Preview https://deploy-preview-917--testcontainers-node.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Anush008
Anush008 commented Mar 5, 2025
View reviewed changes
@cristianrgreco cristianrgreco added maintenance Improvements that do not change functionality patch Backward compatible bug fix labels Mar 5, 2025
@cristianrgreco
Copy link
Copy Markdown
Collaborator

cristianrgreco commented Mar 5, 2025

@Anush008 Tests are failing:

FAIL packages/modules/qdrant/src/qdrant-container.test.ts (17.112 s)
  QdrantContainer
    ✓ should connect to the client (7699 ms)
    ✓ should work with valid API keys (1360 ms)
    ✕ should fail for invalid API keys (1369 ms)
    ✓ should work with config files - valid API key (1352 ms)
    ✕ should work with config files - invalid API key (1332 ms)

  ● QdrantContainer › should fail for invalid API keys

    expect(received).rejects.toThrow(expected)

    Expected substring: "Forbidden"
    Received message:   "Unauthorized"

      at Object.fun [as getCollections] (../../../node_modules/@qdrant/openapi-typescript-fetch/dist/cjs/fetcher.js:172:23)
      at async QdrantClient.getCollections (../../../node_modules/@qdrant/js-client-rest/dist/cjs/qdrant-client.js:835:26)
      at Object.toThrow (../../../node_modules/expect/build/index.js:218:22)
      at Object.<anonymous> (src/qdrant-container.test.ts:45:45)

  ● QdrantContainer › should work with config files - invalid API key

    expect(received).rejects.toThrow(expected)

    Expected substring: "Forbidden"
    Received message:   "Unauthorized"

      at Object.fun [as getCollections] (../../../node_modules/@qdrant/openapi-typescript-fetch/dist/cjs/fetcher.js:172:23)
      at async QdrantClient.getCollections (../../../node_modules/@qdrant/js-client-rest/dist/cjs/qdrant-client.js:835:26)
      at Object.toThrow (../../../node_modules/expect/build/index.js:218:22)
      at Object.<anonymous> (src/qdrant-container.test.ts:74:45)

We should probably keep the version the same as in main, qdrant/qdrant:v1.8.1, otherwise it'll need to be a breaking change. This will be resolved by #867

@Anush008
Copy link
Copy Markdown
Author

Anush008 commented Mar 5, 2025 •
edited
Loading

otherwise it'll need to be a breaking change

In this case, it was actually an update in the REST status message in the latter version.
Qdrant is backwards compatible.

@cristianrgreco cristianrgreco changed the title docs: Demonstrate specifying Qdrant version Update docs to demonstrate specifying Qdrant version Mar 6, 2025
@cristianrgreco cristianrgreco merged commit b6faf5d into testcontainers:main Mar 6, 2025
203 checks passed
vaernion pushed a commit to Arbeidstilsynet/brevgen2 that referenced this pull request Dec 3, 2025
@vaernion
Merged PR 17251: chore(deps): update dependency testcontainers to v10…
796003e 
….20.0

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [testcontainers](https://github.com/testcontainers/testcontainers-node) | devDependencies | minor | [`10.18.0` -> `10.20.0`](https://renovatebot.com/diffs/npm/testcontainers/10.18.0/10.20.0) |

---

### Release Notes

<details>
<summary>testcontainers/testcontainers-node (testcontainers)</summary>

### [`v10.20.0`](https://github.com/testcontainers/testcontainers-node/releases/tag/v10.20.0)

[Compare Source](testcontainers/testcontainers-node@v10.19.0...v10.20.0)

#### Changes

#### 🚀 Features

-   Add support for container commit [@&#8203;smcroskey](https://github.com/smcroskey) ([#&#8203;923](testcontainers/testcontainers-node#923))

#### 🧹 Maintenance

-   Fix flaky healthcheck timeout test [@&#8203;cristianrgreco](https://github.com/cristianrgreco) ([#&#8203;930](testcontainers/testcontainers-node#930))
-   Fix flaky Selenium test [@&#8203;cristianrgreco](https://github.com/cristianrgreco) ([#&#8203;929](testcontainers/testcontainers-node#929))

### [`v10.19.0`](https://github.com/testcontainers/testcontainers-node/releases/tag/v10.19.0)

[Compare Source](testcontainers/testcontainers-node@v10.18.0...v10.19.0)

#### Changes

#### 🚀 Features

-   Add CockroachDB module [@&#8203;leourbina](https://github.com/leourbina) ([#&#8203;910](testcontainers/testcontainers-node#910))

#### 🐛 Bug Fixes

-   Fix lambda container is dropped by ryuk ([#&#8203;848](testcontainers/testcontainers-node#848)) [@&#8203;julienboulay](https://github.com/julienboulay) ([#&#8203;922](testcontainers/testcontainers-node#922))
-   Fix PostgreSQL container restart [@&#8203;cristianrgreco](https://github.com/cristianrgreco) ([#&#8203;914](testcontainers/testcontainers-node#914))

#### 🧹 Maintenance

-   Migrate from Jest to Vitest [@&#8203;cristianrgreco](https://github.com/cristianrgreco) ([#&#8203;921](testcontainers/testcontainers-node#921))
-   GHA workflow job names [@&#8203;cristianrgreco](https://github.com/cristianrgreco) ([#&#8203;920](testcontainers/testcontainers-node#920))
-   Optimise CI to reduce runners usage, caching NPM modules when needed [@&#8203;mdelapenya](https://github.com/mdelapenya) ([#&#8203;918](testcontainers/testcontainers-node#918))
-   Update docs to demonstrate specifying Qdrant version [@&#8203;Anush008](https://github.com/Anush008) ([#&#8203;917](testcontainers/testcontainers-node#917))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually on...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cristianrgreco cristianrgreco cristianrgreco approved these changes

Assignees

No one assigned

Labels

maintenance Improvements that do not change functionality patch Backward compatible bug fix

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Anush008 @cristianrgreco